-
Notifications
You must be signed in to change notification settings - Fork 540
pipeline: outputs: elasticsearch: add cloud_apikey setting #1213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
c92ce19 to
05c8a5c
Compare
|
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
Signed-off-by: Soedarsono <[email protected]>
05c8a5c to
d152079
Compare
|
rebased |
|
@esmerel would you mind reviewing and looking at the conflicts here? |
Signed-off-by: Lynette Miles <[email protected]>
I fixed the conflicts, but it would be good if you doublechecked the code sample from lines 357-374 - I don't know if this is valid or not, so there are two outputs in there. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@soedar The spacing on the code example should be 2 for indenting, not the 4 submitted here.
pipeline/outputs/elasticsearch.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spacing for configuration files should indent 2, not 4.
|
Closed via #1727 |
Updated elasticsearch output docs to include additional cloud_apikey setting.
Ref: fluent/fluent-bit#7935